Disabled knowledge base sources:

• Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.

Walkthrough

This change introduces differentiated deletion handling for hackathon drafts and published hackathons. A new DeleteType discriminant ('draft' | 'hackathon') is added throughout the deletion flow. The hook now routes to the appropriate API endpoint based on type, with optional toast suppression. The page component tracks deletion type and passes it through the hook, enabling delete functionality for both published and draft items.

Changes

Sequence Diagram

sequenceDiagram
    participant User
    participant Page as Page Component
    participant Hook as useDeleteHackathon
    participant DraftAPI as Draft API
    participant HackathonAPI as Hackathon API

    User->>Page: Click delete (type: 'draft' or 'hackathon')
    Page->>Page: Resolve title from data<br/>Populate hackathonToDelete with type
    Page->>Page: Open confirmation dialog
    User->>Page: Confirm deletion
    Page->>Page: Capture title before clearing state
    Page->>Hook: Call deleteHackathon({<br/>type, hackathonId, suppressToast})
    
    alt type === 'draft'
        Hook->>DraftAPI: DELETE /organizations/{id}/hackathons/draft/{id}
        DraftAPI-->>Hook: 204 / error response
    else type === 'hackathon'
        Hook->>HackathonAPI: DELETE /hackathons/{id}
        HackathonAPI-->>Hook: success / error response
    end
    
    Hook-->>Page: Return (isDeleting, error)
    opt suppressToast = false
        Hook->>User: Show success/error toast
    end
    Page->>User: Show success toast with captured title
    Page->>Page: Refetch hackathons data

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related issues

• Issue #416: Directly addressed by adding deleteDraft function in lib/api/hackathons/draft.ts and implementing type-based routing in the deletion hook and page component to resolve draft deletion endpoint mismatch.

Possibly related PRs

• PR #433: Makes identical code-level changes to implement draft deletion routing in hooks/hackathon/use-delete-hackathon.ts and the hackathons page component by introducing DeleteType and conditional endpoint selection.

Suggested reviewers

• 0xdevcollins

Poem

🐰 Hops along with joy so bright,
Drafts and hackathons deleted right!
Routes split true, type guides the way,
Two endpoints dance in deletion's play—
Carrots earned through careful code,
Success toasts mark the rightful road! 🥕

• X
• Mastodon
• Reddit
• LinkedIn